fetcher: Fix previous commit
authorColin Walters <walters@verbum.org>
Tue, 24 Sep 2013 18:10:40 +0000 (14:10 -0400)
committerColin Walters <walters@verbum.org>
Wed, 25 Sep 2013 15:59:43 +0000 (11:59 -0400)
commit7959ad9eaf0055dafb36ec90710d5378fe5714c2
tree39e3686d5e540492ab0dadd4e5321f7a2b2b10b4
parent5f310868f778d3691fcd269718258a8680394a01
fetcher: Fix previous commit

I was getting hangs in the test suite, and looking at the previous
commit, we were calling the async completion functions out of the
finalizer for the URI, which is weird.  I didn't analyze what's going
wrong, but what we really should be doing is processing our internal
queue after we've downloaded a file, and the request is about to be
finalized.

I suspect doing queue management from the finalizer created a circular
reference type situation.

This patch deduplicates the queue processing bits too.

https://bugzilla.gnome.org/show_bug.cgi?id=708126
src/libostree/ostree-fetcher.c